Skip to content

split cron into only 2 tasks#421

Merged
anandkumarpatel merged 5 commits intomasterfrom
SAN-3951-network-canary
Apr 12, 2016
Merged

split cron into only 2 tasks#421
anandkumarpatel merged 5 commits intomasterfrom
SAN-3951-network-canary

Conversation

@anandkumarpatel
Copy link
Copy Markdown

  • added network canary cron task
  • refactored how we do crons, made a main and a canary

Reviewers

Tests

Test any modifications on one of our environments.

  • tested on gamma by anand

Deployment (post-merge)

Ensure that all environments have the given changes.

  • deployed to epsilon
  • deployed to gamma
  • deployed to delta

@anandkumarpatel
Copy link
Copy Markdown
Author

root@gamma-services--10-4-6-251:/home/ubuntu# crontab -l
#Ansible: Khronos CLI - Daily Cleanup
13 1,7,13,19 * * * /opt/runnable/main-cron.sh >> /var/log/main-cron.sh.log 2>&1
#Ansible: Khronos CLI - Canary
*/5 * * * * /opt/runnable/canary-cron.sh >> /var/log/canary-cron.sh.log 2>&1
root@gamma-services--10-4-6-251:/home/ubuntu#

- log-canary.sh
- github-branch-canary.sh
- container-metrics.sh
- { cron_queues: "{{ main_cron_queues }}", script: "main-cron.sh"}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't need quotes around the script name

@anandkumarpatel
Copy link
Copy Markdown
Author

docker run --rm registry.runnable.com/runnable/khronos:SAN-3951-network-canary bash -c " \
  for QUEUE in khronos:canary:build khronos:canary:github-branch khronos:canary:log khronos:canary:network khronos:metrics:container-status; do \
    /khronos/bin/cli.js --queue \$QUEUE --job '{}' --host 10.4.19.17:54321 --username o2mdLh9N9Ke2GzhoK8xsruYPhIQFN7iEL44dQJoq7OM --password wKK7g7NWKpQXEeSzyWB7mIpxZIL8H2mDSf3Q6czR3Vk; \
  done"

@anandkumarpatel anandkumarpatel merged commit ac5fca8 into master Apr 12, 2016
@anandkumarpatel anandkumarpatel deleted the SAN-3951-network-canary branch April 12, 2016 23:33
@bkendall bkendall removed their assignment Jul 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants